Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spi connectivity check for c1101 #1287

Conversation

crasu
Copy link
Contributor

@crasu crasu commented Oct 16, 2022

Description:

I had messed up the wiring from the c1101 to my esp. This will do a basic connectivity check from the esp to the c1101 and print out a warning/success message.

@crasu crasu marked this pull request as draft October 16, 2022 08:41
@crasu crasu force-pushed the spi-connectivity-check-for-c1101 branch 2 times, most recently from b23c9e3 to 17ef9ed Compare October 16, 2022 08:47
@crasu crasu marked this pull request as ready for review October 16, 2022 09:16
main/ZgatewayRF.ino Outdated Show resolved Hide resolved
@crasu crasu closed this Oct 29, 2022
@crasu crasu reopened this Oct 29, 2022
@crasu
Copy link
Contributor Author

crasu commented Oct 29, 2022

Lib is still used for rf gateway

@crasu crasu force-pushed the spi-connectivity-check-for-c1101 branch from 17ef9ed to 817f8fb Compare October 29, 2022 00:58
Signed-off-by: Christian Krämer <christian.kraemer@tngtech.com>
@crasu crasu force-pushed the spi-connectivity-check-for-c1101 branch from 817f8fb to 20f4ead Compare October 29, 2022 08:29
@1technophile
Copy link
Owner

Thanks

@1technophile 1technophile merged commit 567c205 into 1technophile:development Nov 3, 2022
@rds76
Copy link
Contributor

rds76 commented Nov 3, 2022

You should use Flash strings for static Logs and end them with CR
Log.notice(F("C1101 spi Connection OK" CR));

@1technophile
Copy link
Owner

Good catch

@crasu crasu mentioned this pull request Nov 5, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants